Skip to content
This repository was archived by the owner on May 29, 2026. It is now read-only.

feat: implement customizable URL paths for upload endpoints#89

Closed
joseph-sentry wants to merge 2 commits into
mainfrom
feature/customizable-url-paths
Closed

feat: implement customizable URL paths for upload endpoints#89
joseph-sentry wants to merge 2 commits into
mainfrom
feature/customizable-url-paths

Conversation

@joseph-sentry

@joseph-sentry joseph-sentry commented Aug 19, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

This enables downstream projects like prevent-cli to customize upload endpoint
URLs by setting url_paths in the Click context object.

  • Add DEFAULT_URL_PATHS constant in upload.py with configurable URL templates
  • Pass url_paths from command context through do_upload_logic to UploadSender
  • Support different URL paths for COVERAGE and TEST_RESULTS upload types
  • Update prevent-cli main.py to customize TEST_RESULTS path to test_analytics

@joseph-sentry
feat: implement customizable URL paths for upload endpoints
2e74f27 
This enables downstream projects like prevent-cli to customize upload endpoint
URLs by setting url_paths in the Click context object.

- Add DEFAULT_URL_PATHS constant in upload.py with configurable URL templates
- Pass url_paths from command context through do_upload_logic to UploadSender
- Support different URL paths for COVERAGE and TEST_RESULTS upload types
- Update prevent-cli main.py to customize TEST_RESULTS path to test_analytics
@joseph-sentry joseph-sentry force-pushed the feature/customizable-url-paths branch from 7a77495 to 2e74f27 Compare August 19, 2025 15:13
@codecov

codecov Bot commented Aug 19, 2025 •
edited
Loading

Copy link
Copy Markdown

❌ 5 Tests Failed:

Tests completed Failed Passed Skipped
3475 5 3470 5
View the top 3 failed test(s) by shortest run time 
api.temp.calculator.test_calculator::test_divide
Stack Traces | 0.001s run time 
def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError
api.temp.calculator.test_calculator::test_divide
Stack Traces | 0.001s run time 
def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError
api.temp.calculator.test_calculator::test_divide
Stack Traces | 0.001s run time 
def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer
@ElioDiNino ElioDiNino ElioDiNino approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@joseph-sentry @ElioDiNino